Skip to content

refactor(ui5-responsive-popover): API enhancements - #3363

Merged
ilhan007 merged 2 commits into
masterfrom
responsive-popover-api-enhancements
Jun 21, 2021
Merged

ilhan007 merged 2 commits into
masterfrom
responsive-popover-api-enhancements

Conversation

@kskondov

@kskondov kskondov commented May 31, 2021 •

Copy link
Copy Markdown
Contributor

Part of #3107

BREAKING_CHANGE: open method renamed to openBy

@kskondov
kskondov requested review from a team and ilhan007 May 31, 2021 06:34
@ilhan007 ilhan007 changed the title refactor(i5-responsive-popover): API enhancements refactor(ui5-responsive-popover): API enhancements May 31, 2021
@kskondov
kskondov force-pushed the responsive-popover-api-enhancements branch from 7b3f565 to 3599878 Compare May 31, 2021 08:31
@georgimkv

Copy link
Copy Markdown
Contributor

The commit message still says "i5-responsive-popover"

Part of #3107

BREAKING_CHANGE: open method renamed to openBy
@georgimkv
georgimkv force-pushed the responsive-popover-api-enhancements branch from 32fa701 to 5cfd6f4 Compare June 7, 2021 09:39
@georgimkv
georgimkv requested a review from fifoosid June 7, 2021 09:40
@georgimkv georgimkv closed this Jun 8, 2021
@georgimkv georgimkv reopened this Jun 8, 2021

@dimovpetar dimovpetar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace simpleRP.open(btnSimpleRP) with simpleRP.openBy(btnSimpleRP) in ResponsivePopover.html

@georgimkv
georgimkv requested a review from dimovpetar June 18, 2021 11:50
@ilhan007
ilhan007 merged commit 50a568b into master Jun 21, 2021
@ilhan007
ilhan007 deleted the responsive-popover-api-enhancements branch June 21, 2021 08:31
kskondov added a commit that referenced this pull request Jul 5, 2021
Part of #3107

BREAKING_CHANGE: The ResponsivePopover `open` method has been renamed to `openBy`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants